-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve CPU performance #515
base: main
Are you sure you want to change the base?
Improve CPU performance #515
Conversation
Improved CPU performance with: - Merge .position +.rotation to .SetPositionAndRotation - Use TryGetComponent for improved garbage allocations - Simplify string comparisons
Any chance on getting merged? |
Hi @smitdylan2001, sorry for the late reply, there is a chance to be merged yes but we would need a bit more info on the actual gain of your changes. Could you please share:
Thanks!! |
It's not much noticeable at runtime, and I can only benchmark in a few weeks. Here are some of the changes benchmarked: https://forum.unity.com/threads/performance-tips.1336262/ |
If it helps similar changes are made on other Unity repos, like netcode for GO, render pipelines, probuilder and more |
Here's some more info: |
@lucillecaillaud any update on a merge? |
Any updates? |
Improved CPU performance with: